-
Notifications
You must be signed in to change notification settings - Fork 65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sync olm 12-10-21 #223
Sync olm 12-10-21 #223
Conversation
/approve |
* Rename CI github workflows Update the various CI workflows names that are run on a per-PR basis to help distinguish between individual action runs when interacting with the github UI. Signed-off-by: timflannagan <timflannagan@gmail.com> * .github/workflows: Rename the actions that run OLM on kind/minikube and remove any 'e2e' references Signed-off-by: timflannagan <timflannagan@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: d92ef2c06fe3cec1631c15b8877b5b60d3cbd57d
68ba669
to
ecc094a
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: anik120, kevinrizza The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
Update various pkg,test packages are removed any obvious duplicate package imports that violate local static linters. Signed-off-by: timflannagan <timflannagan@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 40b8b7104e3ce577c1c0f0f08e6854d62978169c
…les (#2460) Update the test/e2e/collect-ci-artifacts.sh bash script and extend the list of resources gathered to also create a snapshot of CatalogSource resources in the test namespace. Signed-off-by: timflannagan <timflannagan@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: ea007623445b949ca69667a3c4d52c104595bce0
Signed-off-by: akihikokuroda <akihikokuroda2020@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: e064f3bf6b491b5517fea6fdeba30700e9b4687d
… supplied (#2462) Update the cmd/[catalog,olm] main packages and silence any vet warnings about logrus.Fatal usage that supply formatting directives: ```bash $ go vet ./cmd/... cmd/catalog/main.go:114:3: Fatal call has possible formatting directive %v cmd/olm/main.go:123:3: Fatal call has possible formatting directive %v ``` Signed-off-by: timflannagan <timflannagan@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 3949ceb4dc8cb5668c621904a402de4186f4ea26
Update the CI-related workflows and avoid specifying the paths-ignore keyword. Because we have configure required status contexts through branch protection rules, any workflow that is skipped due to path filtering results in an unknown status, which results in tide stalling any merge despite a PR matching the merge criteria (e.g. lgtm/approval label present). This is a short term fix to the solution: the long term solution is dynamically filtering out certain file paths as a pre-requisite job vs. statically filtering using the paths-ignore filter, while we wait for GitHub to add this as first-class functionality. Signed-off-by: timflannagan <timflannagan@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 285a53507f85ce3509c9dbeb8beeea0482cdf4b6
Signed-off-by: Frederic Giloux <fgiloux@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 53c7d3f3b13e3704cd2d4a7995c7df9c4b7a9dfe
This commit introduces the MAINTAINERS.md file in an effort to decouple project maintainer status from approval and lgtm permissions relied on by the project automated continuous integration tooling. Signed-off-by: Alexander Greene <greene.al1991@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 4f5c16c71fecdfdb02e313cb3173a936261233d5
ecc094a
to
dbd7e9a
Compare
/retest |
3 similar comments
/retest |
/retest |
/retest |
/lgtm |
/retest |
1 similar comment
/retest |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
5 similar comments
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
20 similar comments
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
@anik120: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Note: operator-framework/operator-lifecycle-manager#2216 it yet to make it in this repo due to a failing e2e test. (refer: bz)
This upstream commit being introduced in this PR touches a part of the code that was introduced in the above mentioned PR. The commit's been amended to keep all other changes in it, with the exception of the change in the code that's missing in this repo.